Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Feature] [Frontend] Add about page #571

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

[Feature] [Frontend] Add about page #571

wants to merge 2 commits into from

Conversation

Xiawpohr
Copy link
Collaborator

@Xiawpohr Xiawpohr commented Nov 4, 2024

Summary

  • Add the Page::About containing below section
    • Features introduction
    • Website policy
    • Community information
    • FAQ
  • Rename the componentMobileBottomNav to MainBottomNav
  • Update icons in MainBottomNav
  • Add the component MainSideNav
  • Refactor the component AppLayout
  • Refactor the router

Linked Issue

close #552
close #529

Checklist

  • All new and existing tests pass
  • I have updated the documentation (if applicable)
  • My changes do not introduce new security vulnerabilities

@Xiawpohr Xiawpohr added feature new feature frontend labels Nov 4, 2024
@Xiawpohr Xiawpohr self-assigned this Nov 4, 2024
@github-actions github-actions bot added documentation Improvements or additions to documentation test test improvements labels Nov 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@UranusLin UranusLin Awaiting requested review from UranusLin

@kittybest kittybest Awaiting requested review from kittybest

@DylanYang0523 DylanYang0523 Awaiting requested review from DylanYang0523

At least 1 approving review is required to merge this pull request.

Assignees

@Xiawpohr Xiawpohr

Labels
documentation Improvements or additions to documentation feature new feature frontend test test improvements
Projects
None yet
Milestone
No milestone
1 participant
@Xiawpohr